Skip to content

Change model1#61

Open
Zrt781 wants to merge 2 commits into
mainfrom
change-model
Open

Change model1#61
Zrt781 wants to merge 2 commits into
mainfrom
change-model

Conversation

@Zrt781

@Zrt781 Zrt781 commented Apr 3, 2023

Copy link
Copy Markdown

.env.example

@mingyi77

mingyi77 commented Apr 9, 2023

Copy link
Copy Markdown

/gpt hello

@anc95

anc95 commented May 26, 2023

Copy link
Copy Markdown
Owner

the api key is expired, I just updated, please push new commit to test. enjoy

@ArnavK-09

Copy link
Copy Markdown

/gpt hello

@bhattk64 bhattk64 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Feng-ATM,

Thanks for the PR! I see this is a large refactor with a lot of changes, including:

  • Updates to the .env.example file with new variables.
  • Addition of a new WebAssembly binary file (_tiktoken_bg.wasm).
  • Major rewrite of action/index.cjs with tens of thousands of additions and deletions.

Could you please help by adding a summary to the PR description explaining:

  • The goals of this refactor/change.
  • Why the .wasm file was added and its purpose.
  • Any breaking changes or things maintainers/reviewers should focus on.

Also, it would be helpful to have some inline comments in the code for key parts of the new logic, to make reviewing easier.

Looking forward to your clarifications before diving deeper into the code!

Thanks 🙂

Comment thread yarn.lock
integrity sha512-PuxUbxcW6ZYe656yL3EAhpy7qXKq0DmYsrJLpbB8XrsCP9Nm+XCg9XFMb5vIDliPD7+U/+M+QJlH17XOcB7eXA==
version "7.21.3"
resolved "https://registry.yarnpkg.com/@babel/core/-/core-7.21.3.tgz#cf1c877284a469da5d1ce1d1e53665253fae712e"
integrity sha512-qIJONzoa/qiHghnm0l1n4i/6IIziDpzqc36FBs4pzMhDUraHqponwJLiAKm1hGLP3OSB/TVNz6rMwVGpwxxySw==

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

src/chat.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants